Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(plugin-essentials): support specifying multiple paths in yarn link #4573

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

noahnu
Copy link
Contributor

@noahnu noahnu commented Jun 23, 2022

What's the problem this PR addresses?

I find myself often linking specific packages from a remote monorepo. I don't want to link all the workspaces so I don't use --all. Right now I have to run yarn link <path-1> && yarn link <path-2>. I'd like to be able to run yarn link <path-1> <path-2>. Similar to yarn unlink.

How did you fix it?

We now iterate over all destinations which I've changed to a rest arg in clipanion.

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@merceyz merceyz changed the title feat(plugin-essentials): support multiple workspaces in yarn link feat(plugin-essentials): support multiple workspaces in yarn link Jun 23, 2022
@merceyz merceyz changed the title feat(plugin-essentials): support multiple workspaces in yarn link feat(plugin-essentials): support specifying multiple paths in yarn link Jun 23, 2022
Copy link
Member

@merceyz merceyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, especially since unlink allows specifying multiple paths as well!

@arcanis arcanis merged commit 2ba6e59 into yarnpkg:master Jun 24, 2022
@noahnu noahnu deleted the feat_link_multi branch June 24, 2022 18:22
merceyz pushed a commit that referenced this pull request Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants